Lambda 代码像屎山？三个方法解决！

说 Lambda 写的代码像屎山，其实就是代码不够干净嘛。说到底并不是不会用 Lambda 本身的 API，而是用的方式不对。

Java Lambda 本身提供了非常丰富的方法库，大多数时候我们常用的方法也就为数不多的那几个。

之所以被diss，也大概并不是你用了其中某个不常用的方法（那样别人可能觉得你见多识广）。更多的时候可能是因为这几个原因：

1. 代码写太乱了。一个人说乱那可能是那个人的问题，如果大家都说乱，不好意思，那基本上就是你的问题了。
2. Lambda 函数式本身的写法，尤其是对于从未接触过函数式编程的开发来说，这种写法本身就不太习惯，甚至不喜欢。负面情绪先入为主了，自然就觉得乱了。
3. 还有就是一直被诟病调试问题，Lambda 公认的不便于调试。

先来看一段代码，也就是经常被人（除自己外的所有人）说的屎山代码。

private static List<SimpleUser> dirtyLambda(){
 List<User> userList = User.buildUserList(30);
 List<SimpleUser> simpleUserList = userList.stream()
   .filter(user -> {
    return user.getGender().equals(1)
      && user.getAge() >= 18 && user.getAge() <= 45;
   })
   .map(user -> {
    SimpleUser su = new SimpleUser();
    su.setName(user.getName());
    su.setAge(user.getAge());
    Optional<Address> addressOptional = user.getAddressList().stream()
      .findFirst();
    if (addressOptional.isPresent()) {
     su.setProvince(addressOptional.get().getProvince());
    }
    return su;
   })
   .sorted(Comparator.comparingInt(SimpleUser::getAge))
   .collect(Collectors.toList());
 return simpleUserList;
}

如果不做解释，是不是脏话马上就要出来了。这其实在屎山代码中也最多拍个中等，最起码该换行的换行了，比如那个filter 中的三个并列条件，恐怕你是没见过与或非排列组合的写法，加上不怎么换行，那是真的让人抓狂。

如果你觉得这代码还可以，那有可能你也这么写过。

不瞒各位，这样的代码我曾经写过，而且一天之内不知道写了多少行。曾经有一个需求，一个很复杂的报表，100多个变量+图表+表格，什么最大值、最小值、环比、同比、正序、倒序、top3、top5、top10等等，就是各种能想到的维度统统算一遍。有经验的同学一看就知道，这妥妥的体力活儿啊，但是时间只有一天，没办法，越写越烦躁，直接躺平了，比上面这种更屎的代码一段接一段的写啊。写完别说改了，看都不敢看啊。

说回正题，上面那个代码的逻辑是这样的：

1. 在一个 User列表中筛选男性，且年龄为18到45岁之间的；
2. 然后将 User转换为 SimpleUser类型，获取姓名、年龄，以及地址列表（假设一个人有多个地址）中第一个的省份字段；
3. 然后排序，按照年龄正序排序；
4. 最后返回一个 SimpleUser 列表；

那怎么做才能让代码变得清晰易懂，告别屎山 Lambda 呢？

不管你用什么办法，只要做到下面这3点，Lambda 代码块立马变清晰，最后一点可以适度放宽。

不要超过 5 行

这其实没什么好说的，本身代码规约中就要求最好不写超大方法，也就是行数过多的方法，更何况是在 Lambda 中。在函数式编程中，你写的代码其实是在小括号中，作为参数的形式出现的，一个多行的参数，不敢想啊。

不超过5行可不是说把换行符去掉，把之前的100行直接边 1 行啊。而是下面这样子，stream()就算一行了， 之后每个.function()都算一行，加起来不超过5行。

userList.stream()
  .filter()
  .filter()
  .map()
  .collect(Collectors.toList());

不要一个stream() 后面跟3个filter，4个map，再来个排序，再整个分组，有那么复杂的业务吗，如果有，想想可能在上层设计的时候就出现问题了。

不要出现花括号

不要出现花括号，这其实就是缩短代码行数的一个根本方法。用这个方法，强制你将逻辑抽离出来，这样，你的代码逻辑就会马上变清晰，立竿见影。

拿前面的那端代码举个例子，其中map方法将 User转换为SimpleUser，里面有赋值操作，还有一些判断逻辑。

.map(user -> {
 SimpleUser su = new SimpleUser();
 su.setName(user.getName());
 su.setAge(user.getAge());
 Optional<Address> addressOptional = user.getAddressList().stream()
   .findFirst();
 if (addressOptional.isPresent()) {
  su.setProvince(addressOptional.get().getProvince());
 }
 return su;
})

直接将一段抽取成方法，在 IDEA 中操作也非常方便。选中花括号中的代码，然后右键->Refactor->Extract Method，直接抽取出方法，连名字都帮忙取号了。

同样的，filter()中的三个条件判断也抽离出来。然后效果就是下面这样，每一行的意图都很清晰，谁还会说不能理解。

    private static List<SimpleUser> dirtyLambda(){
        List<User> userList = User.buildUserList(30);
        List<SimpleUser> simpleUserList = userList.stream()
                .filter(user -> filterUser(user))
                .map(user -> getSimpleUser(user))
                .sorted(Comparator.comparingInt(SimpleUser::getAge))
                .collect(Collectors.toList());
        return simpleUserList;
    }

    private static boolean filterUser(User user) {
        return user.getGender().equals(1)
                && user.getAge() >= 18 && user.getAge() <= 45;
    }

    private static SimpleUser getSimpleUser(User user) {
        SimpleUser su = new SimpleUser();
        su.setName(user.getName());
        su.setAge(user.getAge());
        Optional<Address> addressOptional = user.getAddressList().stream()
                .findFirst();
        if (addressOptional.isPresent()) {
            su.setProvince(addressOptional.get().getProvince());
        }
        return su;
    }

最好连 -> 都不要出现

再进一步，就是将 ->也干掉，虽然 ->后面没有花括号已经很简洁了，但是去掉->就不只是简洁了，而是优雅了。

不用->，取而代之的是 ::，最终，去掉->后的代码是下面这样子。

private static List<SimpleUser> dirtyLambda(){
 List<User> userList = User.buildUserList(30);
 return userList.stream()
   .filter(CleanLambda::filterUser)
   .map(CleanLambda::getSimpleUser)
   .sorted(Comparator.comparingInt(SimpleUser::getAge))
   .collect(Collectors.toList());
}

private static boolean filterUser(User user) {
 return user.getGender().equals(1)
   && user.getAge() >= 18 && user.getAge() <= 45;
}

private static SimpleUser getSimpleUser(User user) {
 SimpleUser su = new SimpleUser();
 su.setName(user.getName());
 su.setAge(user.getAge());
 Optional<Address> addressOptional = user.getAddressList().stream()
   .findFirst();
 addressOptional.ifPresent(address -> su.setProvince(address.getProvince()));
 return su;
}

最后

本文只是抛砖引玉，并没有介绍太细节的 Lambda 用法。授人以鱼不如授人以渔，聪明人早就这样写了，更聪明的人已经去改代码了。

往期推荐